Gloas alpha spec 12 changes by eserilev · Pull Request #9596 · sigp/lighthouse

eserilev · 2026-07-08T08:41:30Z

Issue Addressed

All the changes required for gloas alpha spec 12 minus stable containers. Note that EF tests cannot pass without the stable container changes. For example finality tests fail w/ ParentBlockRootMismatch because progressive merkelization isn't implemented.

So I think the plan should be to merge progressive containers, without enabling the new alpha spec 12 tests, and then merge this PR

…pha-spec-12

…into alpha-spec-12

macladson

Just noting that we will need to re-enable the rest of the failing alpha 12 tests which I disabled in #9450

lighthouse/testing/ef_tests/check_all_files_accessed.py

Lines 65 to 70 in e58ec88

    
           # TODO(alpha.12): these tests are disabled until the v1.7.0-alpha.12 EIP-8282 changes are 
        
           # implemented. 
        
           "tests/.*/gloas/fork/.*", 
        
           "tests/.*/gloas/operations/parent_execution_payload/.*", 
        
           "tests/.*/gloas/operations/builder_deposit_request/.*", 
        
           "tests/.*/gloas/operations/builder_exit_request/.*",

mergify · 2026-08-05T06:17:24Z

This pull request has merge conflicts. Could you please resolve them @eserilev? 🙏

pawanjay176

LGTM other than the partial column size enforcement.

mergify · 2026-08-06T00:52:34Z

Queued — the merge queue status continues in this comment ↓.

chong-he

Looks great, just a few small comments

chong-he · 2026-08-06T00:48:26Z

+    #[test]
+    fn test_payload_withdrawals_over_limit() {


There are five types of execution requests and we have two tests checking the execution requests - max_withdrawals_per_payload and max_builder_exit_requests_per_payload.

I understand that the tests are repetitive and so that's probably why you didn't include a test for the other 3 types of requests. So I will leave it to you whether to add them or not.

chong-he · 2026-08-06T00:56:25Z

+    /*
+     * This test ensures that:
+     *
+     * We do not accept gloas blocks whose progressive operation lists exceed their spec limits.


Maybe we can be a bit more specific here? If I understand correctly this is about for Gloas, there can be no more Deposit accepted in a block. A suggestion as below, but feel free to modify as you see fit

Suggested change

* We do not accept gloas blocks whose progressive operation lists exceed their spec limits.

* We do not accept gloas blocks whose progressive operation lists exceed their spec limits (in this case, the `Deposit` should always be empty)

chong-he · 2026-08-06T01:03:17Z

+# 2**6 (= 64) epochs
+MIN_BUILDER_WITHDRAWABILITY_DELAY: 64


I see that the spec has this field:

https://github.com/ethereum/consensus-specs/blame/5366cb59eb39e4ec1d6c468a79cceb626c14c048/configs/mainnet.yaml#L101-L102

# 5000 basis points, 50% of SLOT_DURATION_MS PAYLOAD_DUE_BPS: 5000

Do we need to add it here too? (also the minimal.yaml)

The same goes for the file: common/eth2_network_config/built_in_network_configs/mainnet/config.yaml

mergify · 2026-08-06T04:46:53Z

Merge Queue Status

✅ Entered queue — 2026-08-06 04:46 UTC · Rule: default · triggered by rule Add ready-to-merge labeled PRs to merge queue
🚫 Left the queue — 2026-08-06 04:47 UTC · at 750400ca32dba7d6e8f32ad84dad83b4e23b3be4

This pull request spent 1 minute 10 seconds in the queue, with no time running CI.

Reason

The pull request #9596 has been manually updated

Requeued — the merge queue status continues in this comment ↓.

mergify · 2026-08-06T04:48:43Z

Merge Queue Status

✅ Entered queue — 2026-08-06 04:48 UTC · Rule: default · triggered by rule Add ready-to-merge labeled PRs to merge queue
❌ Checks failed · on draft merge queue: checking unstable (ad8c26f) and #9596 together #9758
🚫 Left the queue — 2026-08-06 05:09 UTC · at 0b9672d2cdf1b5fc68571745bcd244dbdf89fbf2

This pull request spent 20 minutes 28 seconds in the queue, including 18 minutes 32 seconds running CI.

Waiting for

check-success=local-testnet-success
check-success=test-suite-success

All conditions

check-success=local-testnet-success
check-success=test-suite-success
github-review-approved [🛡 GitHub branch protection]
- Gloas alpha spec 12 changes #9596

Reason

The merge conditions cannot be satisfied due to failing checks

test-suite-success

Hint

You may have to fix your CI before adding the pull request to the queue again.
If you update this pull request, to fix the CI, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio queue comment.

Requeued — the merge queue status continues in this comment ↓.

mergify · 2026-08-06T05:06:01Z

Some required checks have failed. Could you please take a look @eserilev? 🙏

eserilev · 2026-08-06T05:36:54Z

@mergify requeue

mergify · 2026-08-06T05:37:17Z

Merge Queue Status

✅ Entered queue — 2026-08-06 05:37 UTC · Rule: default · triggered by @eserilev with the @mergifyio queue command
✅ Checks passed · on draft merge queue: checking unstable (ad8c26f) and #9596 together #9759
✅ Merged — 2026-08-06 06:06 UTC · at 0b9672d2cdf1b5fc68571745bcd244dbdf89fbf2 · squash

This pull request spent 28 minutes 50 seconds in the queue, including 27 minutes 23 seconds running CI.

Required conditions to merge

check-success=local-testnet-success
check-success=test-suite-success
github-review-approved [🛡 GitHub branch protection]
- Gloas alpha spec 12 changes #9596

eserilev requested a review from jxs as a code owner July 8, 2026 08:41

eserilev added blocked gloas labels Jul 8, 2026

impl gloas alpha spec 12 changes

51bbe37

eserilev force-pushed the alpha-spec-12 branch from 609a6cc to 51bbe37 Compare July 8, 2026 08:43

Fix tests

dd0c13d

pawanjay176 mentioned this pull request Jul 9, 2026

Pin genesis generator to devnet-6 #9608

Merged

Merge branch 'unstable' into alpha-spec-12

8410b51

macladson mentioned this pull request Jul 23, 2026

EIP-7688: Forward Compatible Consensus Data Structures #9450

Merged

eserilev added 3 commits August 3, 2026 02:35

Merge branch 'unstable' of https://github.com/sigp/lighthouse into al…

fa281fa

…pha-spec-12

Merge branch 'alpha-spec-12' of https://github.com/eserilev/lighthouse …

38e83ab

…into alpha-spec-12

Merge branch 'unstable' into alpha-spec-12

1fc8af2

eserilev added ready-for-review The code is ready for review work-in-progress PR is a work-in-progress and removed blocked ready-for-review The code is ready for review labels Aug 4, 2026

macladson reviewed Aug 4, 2026

View reviewed changes

final changes

d8aa246

eserilev added ready-for-review The code is ready for review and removed work-in-progress PR is a work-in-progress labels Aug 5, 2026

mergify Bot added waiting-on-author The reviewer has suggested changes and awaits thier implementation. and removed ready-for-review The code is ready for review labels Aug 5, 2026

eserilev added 3 commits August 4, 2026 23:33

Clean up

6b813a2

Merge conflicts

b8781ec

Fix test

30ea172

mergify Bot added ready-for-review The code is ready for review and removed waiting-on-author The reviewer has suggested changes and awaits thier implementation. labels Aug 5, 2026

pawanjay176 approved these changes Aug 6, 2026

View reviewed changes

Comment thread beacon_node/lighthouse_network/src/types/pubsub.rs

chong-he reviewed Aug 6, 2026

View reviewed changes

eserilev added 4 commits August 5, 2026 21:28

size bounds for partial data columns

dcd62c4

Add additional size limit test coverage

31156a4

Update comment

3468d5e

Small spec update

750400c

eserilev added the ready-for-merge This PR is ready to merge. label Aug 6, 2026

FMT

0b9672d

mergify Bot added the queued label Aug 6, 2026

mergify Bot mentioned this pull request Aug 6, 2026

merge queue: checking unstable (ad8c26f) and #9596 together #9758

Closed

8 tasks

mergify Bot added waiting-on-author The reviewer has suggested changes and awaits thier implementation. dequeued and removed ready-for-review The code is ready for review queued labels Aug 6, 2026

mergify Bot added queued and removed dequeued labels Aug 6, 2026

mergify Bot mentioned this pull request Aug 6, 2026

merge queue: checking unstable (ad8c26f) and #9596 together #9759

Closed

8 tasks

mergify Bot merged commit 3c9a9b6 into sigp:unstable Aug 6, 2026
37 of 38 checks passed

mergify Bot removed the queued label Aug 6, 2026

	# TODO(alpha.12): these tests are disabled until the v1.7.0-alpha.12 EIP-8282 changes are
	# implemented.
	"tests/./gloas/fork/.",
	"tests/./gloas/operations/parent_execution_payload/.",
	"tests/./gloas/operations/builder_deposit_request/.",
	"tests/./gloas/operations/builder_exit_request/.",

	* We do not accept gloas blocks whose progressive operation lists exceed their spec limits.
	* We do not accept gloas blocks whose progressive operation lists exceed their spec limits (in this case, the `Deposit` should always be empty)

Uh oh!

Conversation

eserilev commented Jul 8, 2026

Issue Addressed

Uh oh!

macladson left a comment

Choose a reason for hiding this comment

Uh oh!

mergify Bot commented Aug 5, 2026

Uh oh!

pawanjay176 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

mergify Bot commented Aug 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

chong-he left a comment

Choose a reason for hiding this comment

Uh oh!

chong-he Aug 6, 2026

Choose a reason for hiding this comment

Uh oh!

eserilev Aug 6, 2026

Choose a reason for hiding this comment

Uh oh!

chong-he Aug 6, 2026

Choose a reason for hiding this comment

Uh oh!

eserilev Aug 6, 2026

Choose a reason for hiding this comment

Uh oh!

chong-he Aug 6, 2026

Choose a reason for hiding this comment

Uh oh!

eserilev Aug 6, 2026

Choose a reason for hiding this comment

Uh oh!

mergify Bot commented Aug 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge Queue Status

Reason

Uh oh!

mergify Bot commented Aug 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge Queue Status

Reason

Hint

Uh oh!

mergify Bot commented Aug 6, 2026

Uh oh!

eserilev commented Aug 6, 2026

Uh oh!

mergify Bot commented Aug 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge Queue Status

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

mergify Bot commented Aug 6, 2026 •

edited

Loading

mergify Bot commented Aug 6, 2026 •

edited

Loading

mergify Bot commented Aug 6, 2026 •

edited

Loading

mergify Bot commented Aug 6, 2026 •

edited

Loading